fix: Workflow de preview de documentação falha em PRs de fork#97
fix: Workflow de preview de documentação falha em PRs de fork#97camilamaia merged 4 commits intocumbucadev:mainfrom
Conversation
camilamaia
left a comment
camilamaia
left a comment
There was a problem hiding this comment.
Boaaa, ficou ótimo dividir em dois workflows! Vc conseguiu testar no PR para ver se funciona?
Ah, eu devia ter comentado isso na PR antes😅 Consegui testar parcialmente, o build e os artifacts funcionaram certinho no PR do fork. Mas o comentário não apareceu, workflows disparados por Então imagino que essa parte só dá pra validar depois do merge, com um PR interno. |
|
Ahh faz todo sentido! Daria para gente fazer um workflow dispatch para poder acionar manualmente, mas acho que para esse caso é mais fácil testar direto no main mesmo, já que n tem nenhuma implicação grande se não funfar. Bora dale então! |
2 participants
O que foi alterado
O workflow de preview da documentação foi dividido em dois:
deploy-preview-documentation.yml: faz o build, salva o número do PR como artifact e continua publicando o preview apenas para PRs internos.comment-preview-documentation.yml: novo workflow responsável por comentar no PR após o build, usandoworkflow_run.O comentário agora é adaptado à origem do PR:
Além disso, a lógica de comentário continua atualizando o comentário existente em vez de criar vários.
Motivo da mudança
PRs de forks recebem um
GITHUB_TOKENcom permissões limitadas. Isso fazia o deploy paragh-pagesfalhar e também impedia o comentário no PR com erro 403.Separar o fluxo em dois workflows segue o padrão recomendado para esse cenário: o
pull_requestfaz apenas o processamento seguro do PR, e oworkflow_runfaz a etapa que precisa de escrita no PR.Issue relacionada
Resolve #93
Referência